Skip to content

[CI][Benchmarks] Archive cutoff date #19514

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 30, 2025

Conversation

PatKamin
Copy link
Contributor

@PatKamin PatKamin commented Jul 18, 2025

Archived runs older than 3 times the specified days are not included in the dashboard, i.e.. when archiving data older than 7 days, runs older than 21 days are not included.

This change will prevent the archived data file used in the dashboard from a limitless size grow.

@PatKamin PatKamin requested a review from a team as a code owner July 18, 2025 10:56
@PatKamin PatKamin changed the title Archive cutoff date [CI][Benchmarks] Archive cutoff date Jul 18, 2025
@PatKamin PatKamin force-pushed the archive-cutoff-date branch from 3b87887 to 503b0c0 Compare July 18, 2025 11:00
@PatKamin PatKamin force-pushed the archive-cutoff-date branch from 503b0c0 to 127d14f Compare July 29, 2025 12:06
@PatKamin PatKamin requested a review from pbalcer July 29, 2025 12:06
PatKamin added 2 commits July 30, 2025 07:38
Load and parse results only up to the set number of days old which
is three times the defined archiving times.

Archived runs older than 3 times the specified days are not included in the dashboard,
ie. when archiving data older than 7 days, runs older than 21 days are not included.
Both 'stdout' and 'passed' fields were recently removed fro mthe Result class
@PatKamin
Copy link
Contributor Author

@intel/llvm-gatekeepers, please merge

@steffenlarsen steffenlarsen merged commit 8f54710 into intel:sycl Jul 30, 2025
38 of 40 checks passed
@PatKamin PatKamin deleted the archive-cutoff-date branch July 30, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants